-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: add more clear and description comments #53
docs: add more clear and description comments #53
Conversation
@hallazzang I added/updated comments in both |
…51-refactor-comments * commit '3bc1decf7cc852747e275db20404446c69d4ede9': chore: bump Cosmos SDK version to 0.44.2 test: update simulation tests (tendermint#54)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM,
but It seems like proto-format proto-lint proto-gen
was not executed well.
This can happen if the proto docker container for another module collides even if the make is executed. let me commit this.
and Swagger was updated through this PR. The API specification is not changed but description, etc., so it is not necessary to upgrade the swagger version, but if there is a change from next time, it would be good to upgrade the swagger version from client/docs/config.json
, re-generate it, and update it with the swagger hub.
Description
closes: #51
Tasks
client
folder to improve commentskeeper
folder to improve commentsscan through(work will be done in test: randomize simulations for edge cases #32)simulation
folder to improve commentstypes
folder to improve commentsBefore we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes